Skip to content

Conversation

@hivanalejandro
Copy link
Contributor

@hivanalejandro hivanalejandro commented Apr 1, 2025

Description

Fixes b/372570764

Changes in documentation: cl/742799018

Note: If you are not associated with Google, open an issue for discussion before submitting a pull request.

Checklist

Readiness

  • Yes, merge this PR after it is approved
  • No, don't merge this PR after it is approved

Style

Testing

Intended location

API enablement

  • If the sample needs an API enabled to pass testing, I have added the service to the Test setup file

Review

  • If this sample adds a new directory, I have added codeowners to the CODEOWNERS file

@hivanalejandro hivanalejandro requested a review from glasnt April 4, 2025 16:42
@glasnt glasnt marked this pull request as ready for review April 6, 2025 22:44
@glasnt glasnt requested review from a team as code owners April 6, 2025 22:44
Copy link
Contributor

@glasnt glasnt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/gcbrun

@glasnt
Copy link
Contributor

glasnt commented Apr 6, 2025

you can't run /gcbrun in an approval? (trying it in a comment to re-run)

@glasnt glasnt merged commit 1cfc3de into terraform-google-modules:main Apr 6, 2025
5 checks passed
niharika-98 pushed a commit to niharika-98/terraform-docs-samples that referenced this pull request Sep 7, 2025
…rraform-google-modules#833)

* docs: clarify placeholder values in custom_domain_mapping example

* docs: clarify placeholder usage without breaking sample

---------

Co-authored-by: Katie McLaughlin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants